Skip to content

Hotfix/java home#33

Merged
zechtz merged 3 commits intomainfrom
hotfix/java-home
Dec 1, 2025
Merged

Hotfix/java home#33
zechtz merged 3 commits intomainfrom
hotfix/java-home

Conversation

@zechtz
Copy link
Owner

@zechtz zechtz commented Dec 1, 2025

No description provided.

- Filter out JAVA_HOME and PATH from base environment before setting
- Determine single JAVA_HOME value based on precedence rules
- Set JAVA_HOME and PATH only once to avoid conflicts
- Ensures service-level JAVA_HOME properly overrides profile-level setting

This fixes the issue where both profile and service JAVA_HOME appeared
in environment variables, causing the first (wrong) one to be used.
- Change verbose flag insertion to replace wrapper command instead of goal
- Add -X flag right after ./mvnw for Maven (not after spring-boot:run)
- Add -i flag right after ./gradlew for Gradle (not after bootRun)
- Works correctly regardless of whether javaOpts are present

This fixes the issue where verbose flags weren't being added to the
command because the string replacement was looking for a pattern that
didn't match when javaOpts were present.

Example output:
- Before: cd /path && ./mvnw spring-boot:run
- After:  cd /path && ./mvnw -X spring-boot:run
Fixed three critical issues preventing the verbose logging checkbox from persisting:
- Added VerboseLogging field to ServiceConfigRequest model
- Updated frontend payload to include verboseLogging when saving services
- Fixed loadDynamicServices to load verbose_logging column from database

The verbose logging feature now works end-to-end:
1. User checks "Enable verbose logging" checkbox
2. Value is sent in save payload to backend
3. Backend saves to database
4. Value is loaded when services are fetched
5. Checkbox shows correct state when reopening service config
6. Maven services run with -X flag, Gradle services with -i flag
@zechtz zechtz merged commit e81132c into main Dec 1, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant